Skip to content

fix: sample_random_orientation_z passes 1-D axis to Nx3 sampler - #15

Draft
andrewwhitecdw wants to merge 1 commit into
NVlabs:mainfrom
andrewwhitecdw:bugfix/utilities-86d47b10
Draft

fix: sample_random_orientation_z passes 1-D axis to Nx3 sampler#15
andrewwhitecdw wants to merge 1 commit into
NVlabs:mainfrom
andrewwhitecdw:bugfix/utilities-86d47b10

Conversation

@andrewwhitecdw

@andrewwhitecdw andrewwhitecdw commented Jul 27, 2026

Copy link
Copy Markdown

Problem

fix: sample_random_orientation_z passes 1-D axis to Nx3 sampler

Fix

Replace:

    z_axis = sample_spherical_cap(
        mean_axis_z, axis_cone_aperture, num_samples_per_dir=1
    )[0]

with:

    z_axis = sample_spherical_cap(
        np.atleast_2d(mean_axis_z), axis_cone_aperture, num_samples_per_dir=1
    )[0]

Files changed

  • graspsampling-py-defgraspsim/graspsampling/utilities.py

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant